Skip to content

Auto assign workflow#1276

Closed
tomassrnka wants to merge 8 commits into
mainfrom
auto-assign-workflow
Closed

Auto assign workflow#1276
tomassrnka wants to merge 8 commits into
mainfrom
auto-assign-workflow

Conversation

@tomassrnka
Copy link
Copy Markdown
Member

@tomassrnka tomassrnka commented Oct 1, 2025

Fix when we actually run the workflow, so edits/drafts do not trigger it.


Note

Limits the pull_request_target triggers in .github/workflows/auto-request-same-site.yml to only opened and ready_for_review.

Written by Cursor Bugbot for commit 1b1c1ad. This will update automatically on new commits. Configure here.

@e2b-request-same-site-reviewers e2b-request-same-site-reviewers Bot requested review from ValentaTomas and removed request for ValentaTomas October 1, 2025 10:37
@tomassrnka tomassrnka closed this Oct 1, 2025
# Works for PRs from branches and forks (runs in base-repo context)
pull_request_target:
types: [opened, reopened, ready_for_review, synchronize, edited]
types: [opened, ready_for_review]
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: PR Reopen and Sync Events Fail to Trigger Workflow

The workflow no longer triggers on reopened or synchronize events. This means reopened PRs won't get reviewer assignments, and new commits won't update existing assignments, potentially leaving new code unreviewed. This seems inconsistent with the goal of avoiding triggers on "edits/drafts," as reopened and synchronize are not related to those scenarios.

Fix in Cursor Fix in Web

@jakubno jakubno deleted the auto-assign-workflow branch October 1, 2025 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant